Skip to content

Jira 797, Central can scan with MAC address, git 376 #416

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

SidLeung
Copy link
Contributor

New feature:

  • Add the capability for a Central to scan for a
    Peripheral using its MAC address.

Code Modifications:

  • Added the API for MAC address scanning.
  • For Device Manager, added provision for holding the
    MAC address of a Peripheral.
  • For scan discovery handler, added the checking
    for MAC address comparison, if MAC address scanning
    is requested.

File changes:

  1. libraries/CurieBLE/src/BLEDevice.cpp:
    • Added API for MAC address scanning.
  2. libraries/CurieBLE/src/BLEDevice.h:
    • API definition.
  3. libraries/CurieBLE/src/internal/BLEDeviceManager.cpp:
    • Added provision for storing the MAC address.
    • Added MAC address comparison in the scan discovery
      handle if MAC address scanning is selected.
  4. libraries/CurieBLE/src/internal/BLEDeviceManager.h:
    • Prototyping.

New feature:
-  Add the capability for a Central to scan for a
   Peripheral using its MAC address.

Code Modifications:
-  Added the API for MAC address scanning.
-  For Device Manager, added provision for holding the
   MAC address of a Peripheral.
-  For scan discovery handler, added the checking
   for MAC address comparison, if MAC address scanning
   is requested.

File changes:

1. libraries/CurieBLE/src/BLEDevice.cpp:
   - Added API for MAC address scanning.
2. libraries/CurieBLE/src/BLEDevice.h:
   - API definition.
3. libraries/CurieBLE/src/internal/BLEDeviceManager.cpp:
   - Added provision for storing the MAC address.
   - Added MAC address comparison in the scan discovery
     handle if MAC address scanning is selected.
4. libraries/CurieBLE/src/internal/BLEDeviceManager.h:
   - Prototyping.
@SidLeung
Copy link
Contributor Author

@bigdinotech @eriknyquist , please review the code change. @noelpaz @russmcinnis , please note that this PR is originated from PR #397. If you have tested it against Jira #797, please wait for code approval to perform final verification prior to merging.

@SidLeung
Copy link
Contributor Author

SidLeung commented Feb 1, 2017

@noelpaz @russmcinnis , please confirm if this PR passed verification testing.

@SidLeung
Copy link
Contributor Author

SidLeung commented Feb 3, 2017

@noelpaz , @russmcinnis , please provide feedback to close out this PR.

@noelpaz
Copy link
Contributor

noelpaz commented Feb 7, 2017

It passed verification testing

@SidLeung
Copy link
Contributor Author

SidLeung commented Feb 7, 2017

@yashaswini-hanji @calvinatintel , would you please merge this PR to the main trunk. It has gone through code approval and testing. Please close the PR upon merge completion.

@yashaswini-hanji
Copy link
Contributor

Changes merged to base branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants